-
Notifications
You must be signed in to change notification settings - Fork 72
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(jans-linux-setup): top level wrapper command jans #8737
Conversation
Signed-off-by: Mustafa Baser <mbaser@mail.com>
Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.
Note 🟢 Risk threshold not exceeded. Change Summary (click to expand)The following is a summary of changes in this pull request made by me, your security buddy 🤖. Note that this summary is auto-generated and not meant to be a definitive list of security issues but rather a helpful summary from a security perspective. Summary: The code changes in this pull request are focused on updates to the configuration and setup scripts for the Jans Linux setup. The changes include:
Files Changed:
Powered by DryRun Security |
|
Hi @devrimyatar As a result of this PR merge, I was expecting |
It seems this is not possible. You have to exit from current bash and open new one. This is explaine in stackoverflow You can't do it from python, but some clever bash tricks can do something similar. The basic reasoning is this: environment variables exist in a per-process memory space. When a new process is created with fork() it inherits its parent's environment variables. |
Hmm. Let me check. |
Unfortunately, same result even with new bash session. |
closes #8697
In this PR a command
jans
is located under directory/opt/jans/bin
and this directory is added to$PATH
so that command is available without full path. To see help,